Skip to content

[stable-3.12] Fix contacts backup job interval#6144

Merged
AndyScherzinger merged 1 commit into
stable-3.12from
backport/6139/stable-3.12
May 25, 2020
Merged

[stable-3.12] Fix contacts backup job interval#6144
AndyScherzinger merged 1 commit into
stable-3.12from
backport/6139/stable-3.12

Conversation

@backportbot-nextcloud

@backportbot-nextcloud backportbot-nextcloud Bot commented May 25, 2020

Copy link
Copy Markdown

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #6139

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Unit test failed, but no output was generated. Maybe a preliminary stage failed.

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Lint

Typestable-3.12PR
Warnings9494
Errors11

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings10
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings77
Security Warnings44
Dodgy code Warnings142
Total381

SpotBugs (stable-3.12)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings10
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings77
Security Warnings44
Dodgy code Warnings142
Total381

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Job interval was set in ms, when API requred minutes.
Use of invalid time unit resulted in period of ~164 years.

This change removes ms interval constants and sets
proper contact backup job period.

Fixes #6041

Signed-off-by: Chris Narkiewicz <hello@ezaquarii.com>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@tobiasKaminsky
tobiasKaminsky force-pushed the backport/6139/stable-3.12 branch from 75c3da3 to 5ff4856 Compare May 25, 2020 11:12
@tobiasKaminsky

Copy link
Copy Markdown
Member

warning This backport had conflicts and is incomplete warning

Resolved :+1

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/14150.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Unit test failed, but no output was generated. Maybe a preliminary stage failed.

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Lint

Typestable-3.12PR
Warnings9494
Errors11

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings10
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings77
Security Warnings44
Dodgy code Warnings142
Total381

SpotBugs (stable-3.12)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings10
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings77
Security Warnings44
Dodgy code Warnings142
Total381

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

@AndyScherzinger
AndyScherzinger merged commit ad25c8d into stable-3.12 May 25, 2020
@delete-merged-branch
delete-merged-branch Bot deleted the backport/6139/stable-3.12 branch May 25, 2020 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants